Started running into this one, would be great to get it merged.

Seeing this as well. 🙏

We're seeing this too. Would be awesome to get this in soon

bumping again, this is causing errors now:
Could not log "sql.active_record" event. NoMethodError: undefined method 'sql_runtime' for module ActiveRecord::RuntimeRegistry on Rails 8.1.0 which was released yesterday

I created a monkeypatch to fix this in the meantime, based on @rubyroob's excellent work in #249.

Stick this in a file in config/initializers and it should clear up issues until the gem is updated.

# frozen_string_literal: true

LAST_TESTED_VERSION = '4.18.0'

require 'rails_semantic_logger/version'

unless RailsSemanticLogger::VERSION == LAST_TESTED_VERSION
  raise "rails_semantic_logger is version #{RailsSemanticLogger::VERSION} but the monkey patch was last tested on " \
        "#{LAST_TESTED_VERSION} - manually check if it can find the sql_runtime module."
end

module RailsSemanticLogger
  module ActiveRecord
    class LogSubscriber < ActiveSupport::LogSubscriber
      def self.runtime=(value)
        if ::ActiveRecord::RuntimeRegistry.respond_to?(:stats)
          ::ActiveRecord::RuntimeRegistry.stats.sql_runtime = value
        else
          ::ActiveRecord::RuntimeRegistry.sql_runtime = value
        end
      end

      def self.runtime
        if ::ActiveRecord::RuntimeRegistry.respond_to?(:stats)
          ::ActiveRecord::RuntimeRegistry.stats.sql_runtime ||= 0
        else
          ::ActiveRecord::RuntimeRegistry.sql_runtime ||= 0
        end
      end
    end
  end
end

@zzak
Hello, Thank you so much for submitting this pull request! I know you must be busy, but I would really appreciate it if you could merge this when you have a free moment. Thank you for your contribution.

I created a monkeypatch to fix this in the meantime, based on @rubyroob's excellent work in #249.

It's a shame your tag didn't work, as I started seeing this error today - thanks @orehmane !

Thank you @rubyroobs! Your patch saved me earlier, just trying to pay it forward. :)

@reidmorrison would it be possible to have it merged sooner rather than later given that rails 8.1 is out?

@reidmorrison more and more people are going to start hitting this on rails 8.1+. This patch is important to have.